Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

gEconpy v2.0 #38

Open
wants to merge 101 commits into
base: main
Choose a base branch
from
Open

gEconpy v2.0 #38

wants to merge 101 commits into from

Conversation

jessegrabowski
Copy link
Owner

@jessegrabowski jessegrabowski commented Oct 15, 2024 •
edited by github-actions bot
Loading

Massive refactor.

📚 Documentation preview 📚: https://gEconpy--38.org.readthedocs.build/en/38/

@review-notebook-app Review Notebook App
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@codecov Codecov
Copy link

codecov bot commented Oct 16, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 76.98201% with 691 lines in your changes missing coverage. Please review.

Project coverage is 72.09%. Comparing base (5ab3aa5) to head (c8d01a6).

Files with missing lines Patch % Lines
gEconpy/_version.py 40.66% 213 Missing ⚠️
gEconpy/model/statespace.py 31.55% 167 Missing ⚠️
gEconpy/model/model.py 84.64% 84 Missing ⚠️
gEconpy/model/steady_state.py 73.87% 29 Missing ⚠️
gEconpy/numbaf/utilities.py 51.66% 29 Missing ⚠️
gEconpy/parser/parse_distributions.py 77.11% 27 Missing ⚠️
gEconpy/model/perturbation.py 85.71% 25 Missing ⚠️
gEconpy/solvers/gensys.py 88.19% 19 Missing ⚠️
gEconpy/model/compile.py 86.23% 15 Missing ⚠️
gEconpy/utilities.py 81.33% 14 Missing ⚠️
... and 13 more
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #38      +/-   ##
==========================================
- Coverage   79.96%   72.09%   -7.88%     
==========================================
  Files          36       33       -3     
  Lines        5121     5662     +541     
==========================================
- Hits         4095     4082      -13     
- Misses       1026     1580     +554
Files with missing lines Coverage Δ
gEconpy/classes/containers.py 96.23% <100.00%> (-0.84%) ⬇️
gEconpy/classes/time_aware_symbol.py 93.54% <100.00%> (-3.18%) ⬇️
gEconpy/model/parameters.py 100.00% <100.00%> (ø)
gEconpy/numbaf/__init__.py 100.00% <100.00%> (ø)
gEconpy/numbaf/intrinsics.py 31.37% <ø> (ø)
gEconpy/parser/constants.py 100.00% <ø> (ø)
gEconpy/parser/gEcon_parser.py 98.48% <100.00%> (+0.92%) ⬆️
gEconpy/parser/parse_plaintext.py 98.52% <100.00%> (-1.48%) ⬇️
gEconpy/parser/validation.py 83.33% <100.00%> (ø)
gEconpy/solvers/shared.py 100.00% <100.00%> (ø)
... and 23 more

@jessegrabowski
Disable partial steady state when method = root
2ef2aac 
Begin refactor of perturbation solver
@jessegrabowski
Work on perturbation solution
32bfdeb
@jessegrabowski
Add logger
970ccf5
@jessegrabowski
Signature of function generated by numba_lambdify matches `sp.lambd…
3a754e9 
…ify`
@jessegrabowski
Add first-order linearization functions to model
45148b2
@jessegrabowski
Rebase main, run new pre-commit linter
f49b3b4
@jessegrabowski
post-merge bugfixes
8b7a37c
@jessegrabowski
Add example notebook about TimeAwareSymbol
fb8c437
@jessegrabowski
Bug fixes and tests of steady state functions
bf6a5aa
@jessegrabowski
Progress on refactor
16ecc65
@jessegrabowski
More progress towards full refactor
be701e6
@jessegrabowski
Bug fixes and expand test coverage
5586a6a
@jessegrabowski
Refactor steady state tests
6ce2aa3
@jessegrabowski
Allow partial steady state when how="root"
780d030
@jessegrabowski
Refactor stationary covariance and autocovariance functions
647952e
@jessegrabowski
Using logging instead of print statements
6e3cf87
@jessegrabowski
Partially njit gensys solver
19a8ffc
@jessegrabowski
Set compile arguments so all backends return are identical
322e972
@jessegrabowski
Move build_Q_matrix and compute_autocorrelation_matrix functions
bc5ab7c
@jessegrabowski
More tests
e2480ab
@jessegrabowski
Implement irf and simulation, early work on plotting refactor
9ecfe4c
@jessegrabowski
Add builder to compile model to PyMCStateSpace
836d5e5
@jessegrabowski
Add pytensor rational expectations solvers
fea400a 
Move all gensys-related functions to gensys.py

Move all cycle_reduction related functions to cycle_reduction.py
@jessegrabowski
Small adjustments
6d3417d
@jessegrabowski
Rename bk_condition to check_bk_condition
bedf1e5 
Simplify matrix elements
@jessegrabowski
Greatly reduce default iterations of scan cycle reduction
d84b378
@jessegrabowski
Include steady-state equations when checking for orphan/extra parameters
ccf9f60
@jessegrabowski
Fix bug in IRF
12ffd88
@jessegrabowski
Add check for sympy float/int when converting to pytensor
c9acbb6
@jessegrabowski
Allow definitions to refer to other definitions
33d1929
@jessegrabowski
Improve simplification via tryreduce
fb29ab3
@jessegrabowski
Remove tryreduce from test GCNs
b40c299
@jessegrabowski
Statespace doesn't expect deterministic parameters
648e96c
@jessegrabowski
Fix tests
41b3d99
@jessegrabowski
Allow user to define parameters that are only used by other parameters
91c48e3
@jessegrabowski
Save more BK information in statespace model
8ddd69a
@jessegrabowski
Simplify eigenvalue computation
2ed79ca
@jessegrabowski
Update sumary test
c128025
@jessegrabowski
Add Baxter_King_1993.gcn
0f3d297
@jessegrabowski
Clean up complete example
386c2c4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jessegrabowski